Skip to content

Add changelog for #4024 and update register_page docstrings - #4034

Merged
T4rk1n merged 2 commits into
plotly:devfrom
AnnMarieW:update-changelog
Oct 5, 2026
Merged

T4rk1n merged 2 commits into
plotly:devfrom
AnnMarieW:update-changelog

Conversation

@AnnMarieW

@AnnMarieW AnnMarieW commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator

No code changes - just add changelog and updated docstring for PR #4024

Does this need to be updated too? https://github.com/plotly/dash/blob/dev/.ai/ARCHITECTURE.md#dash-constructor-parameters

@sonarqubecloud

sonarqubecloud Bot commented Oct 2, 2026

Copy link
Copy Markdown

@T4rk1n

T4rk1n commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

No code changes - just add changelog and updated docstring for PR #4024

Does this need to be updated too? https://github.com/plotly/dash/blob/dev/.ai/ARCHITECTURE.md#dash-constructor-parameters

Yes, the AI documentation should follow, altho I am not sure of listing all the arguments like that is really useful and the document is getting too big. I'd vote to actually remove that section, AI can grep the constructor easily and it can drift.

Instead I'd improve this section: https://github.com/plotly/dash/blob/dev/.ai/ARCHITECTURE.md#meta-tag-sanitization with the details how it works.

@AnnMarieW

Copy link
Copy Markdown
Collaborator Author

Should we just leave the AI docs update for a different PR?

@T4rk1n

T4rk1n commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Should we just leave the AI docs update for a different PR?

Yes, I created #4051 to track it.

@T4rk1n T4rk1n left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💃

@T4rk1n
T4rk1n merged commit fdedbbf into plotly:dev Oct 5, 2026
37 of 38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants